-
Notifications
You must be signed in to change notification settings - Fork 73
Asyncio Module Compact Serialization [6/6] #752
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Fixed a bug where an asyncio Lock was tried to be acquired without a release - Store asyncio tasks in order not to lose them - Removed start/shutdown from AsyncioReactor
…integration-tests2
…integration-tests2
emreyigit
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Biggest thing in the PR is compact tests. And, it looked same to me compared to existing compact tests.
# Conflicts: # hazelcast/internal/asyncio_connection.py # hazelcast/internal/asyncio_reactor.py
* Removed unnecessary client components * Converted partition service to async * Ported compact compatibility test
|
@ihsandemir @emreyigit I've added some more code to this PR at: I've removed unused components of the client, ported partition service, and ported compact compatibility tests. Please add review comments if necessary. |
No description provided.